-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Correcting BME280 IIR filter setting #1787
Conversation
I'm still working on getting a docker setup running smoothly (very limited time here), but I'll be happy to test when it hits the dev branch and I can use nodemcubuild.com |
I've gotten docker working and can confirm that this fixes the bug in the BME280 module. Pressures are now correct when the IIR filter parameter is 1. It's difficult to accurately test the filter itself, but turning the filter off (0), vs using a filter coefficient of 16 (4) show clear differences in data variability so it appears the filter is working correctly. |
Feel free to leave feedback with https://github.com/marcelstoer/docker-nodemcu-build (assuming you're using that image) if you think documentation should be improved. |
@marcelstoer the docker side of things worked pretty well, though its inability to run on Windows 7 is a real shame. What irked me more is that it's rather tedious to build, and difficult to figure out. The section on building firmware in the Nodemcu docs is very brief. The location for user_modules.h and user_config.h are not mentioned, and this isn't covered at all from your docker repository's Readme. You might get more individuals interested in helping out if this was more accessible and well-documented. Editing user_modules.h, user_version.h, and user_config.h (and sometimes more) each time you want to build a new firmware is also rather tedious. Why not have all of that in one file? Better yet would be a way to preserve multiple build configurations (e.g. different sets of modules) without maintaining multiple sets of the user_*****.h files, especially if one could do that via scripting or something. I find it quite useful that your nodemcu-build.com adds the commit, current date, and included modules into the firmware's version info string. This is really the only way to see which modules a firmware contains. I'm surprised that there's no built-in support for this when building via docker. I've already added the build date automatically, but again this just seems like something that should be built-in at least as an option. |
@FrankX0: Yes, you are right. There was a bug in the code. I was misled by the fact that Bit 1 does not have any meaning. The documentation on page 28 says: Table 26: Register 0xF5 “config”
I think we can merge. |
True, could you maybe document what you were missing and prepare a PR to that effect?
There's some info at https://github.com/nodemcu/nodemcu-firmware#build-options but I agree it doesn't really belong there (historical left-over).
Indeed but I argue it doesn't belong there but here.
Yep #1223.
I am surprised I had to build that myself for the cloud builder. Again, I argue it should be a standard firmware feature; #1739. |
Thanks Frank. |
Fixes #1786.
dev
branch rather than formaster
.docs/en/*
.Parameters "inactive_duration" and "IIR_filter" were incorrectly written to the config register.
@mickley: can you test?
@vsky279: do you agree?